Skip to content

Recognize CMA as a valid center in rd_grib2.F#285

Open
Geek-007 wants to merge 1 commit into
wrf-model:developfrom
Geek-007:master
Open

Recognize CMA as a valid center in rd_grib2.F#285
Geek-007 wants to merge 1 commit into
wrf-model:developfrom
Geek-007:master

Conversation

@Geek-007

@Geek-007 Geek-007 commented Apr 14, 2026

Copy link
Copy Markdown

Include support for CMA dataset.

@jimbresch jimbresch changed the title Add support for CMA source in rd_grib2.F Recognize CMA as a valid center in rd_grib2.F Apr 14, 2026
@jimbresch

Copy link
Copy Markdown
Collaborator

This small change recognizes CMA as a valid center for labeling purposes. It was already added to g2print, but for some reason was omitted from rd_grib2.F. It can be merged once the unnecessary comment is removed.
If it can be confirmed that iprocess 15 is the CMA GFS Model (rather than another model or analysis) the new code can be changed to

else if (icenter .eq. 38) then
map%source = 'CMA'
if (iprocess.eq.15) then
map%source = 'CMA GFS Model'
endif

@Geek-007

Copy link
Copy Markdown
Author

Thanks for adding that.

@islas

islas commented Jun 27, 2026

Copy link
Copy Markdown
Contributor

@jimbresch

It can be merged once the unnecessary comment is removed

Do you mean the ! Edited by ... comment?

@jimbresch

Copy link
Copy Markdown
Collaborator

Yes, we don't normally include 'edited by' or 'modified by' comments in the code. The pull request serves as the attribution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants